Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Leave notes on service worker script fetching for V1 #1450

Merged
merged 1 commit into from
Jul 19, 2019

Conversation

jungkees
Copy link
Collaborator

@jungkees jungkees commented Jul 15, 2019

The Update algorithm step 9 has two issues:

  • Using to-be-created environment settings object instead of a concrete
    environment settings object
  • Passing null value of job's client to the fetch a script algorithms in HTML

This change leaves notes on these issues to track them in the nightly version
and moves them to the Version 2 milestone in the github repository.

For #1013.


Preview | Diff

The Update algorithm step 9 has two issues:
 - Using to-be-created environment settings object instead of a concrete
 environment settings object
 - Passing null value of job's client to the fetch a script algorithms in HTML

This change leaves notes on these issues to track them in the nightly version
and moves them to the Version 2 milestone in the github repository.

For #1013.
@jakearchibald jakearchibald merged commit 2e2d872 into master Jul 19, 2019
@jakearchibald jakearchibald deleted the note-on-script-fetching-in-update branch July 19, 2019 09:59
jakearchibald pushed a commit to CYBAI/ServiceWorker that referenced this pull request Aug 9, 2019
The Update algorithm step 9 has two issues:
 - Using to-be-created environment settings object instead of a concrete
 environment settings object
 - Passing null value of job's client to the fetch a script algorithms in HTML

This change leaves notes on these issues to track them in the nightly version
and moves them to the Version 2 milestone in the github repository.

For w3c#1013.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants